Skip to content

fix: CIエラーを修正 - 型安全性と依存関係の問題を解決#21

Merged
at-ok merged 2 commits into
mainfrom
claude/codebase-review-refactor-014RfXjs8uCCm3sMAL4vijND
Nov 20, 2025
Merged

fix: CIエラーを修正 - 型安全性と依存関係の問題を解決#21
at-ok merged 2 commits into
mainfrom
claude/codebase-review-refactor-014RfXjs8uCCm3sMAL4vijND

Conversation

@at-ok

@at-ok at-ok commented Nov 20, 2025

Copy link
Copy Markdown
Owner

修正内容

1. Supabase createClient関数のエクスポート

  • lib/supabase.ts に createClient 関数を追加
  • APIルートで使用できるように再エクスポート

2. 型安全性の改善

  • app/api/push/send/route.ts: SubscriptionRow インターフェースを定義
  • app/api/push/subscribe/route.ts: 型キャストを適切に実装
  • app/gamemaster/page.tsx: filter/map関数の引数に明示的な型を追加
  • app/chaser/page.tsx: map関数の引数に明示的な型を追加

3. テストの修正

  • hooks/tests/useLocation.test.ts: resumeGame を削除(存在しない関数)
  • hooks/tests/useLocationHistory.test.tsx: calculateDistance を lib/geometry から直接インポート

4. 通知APIの型問題を解決

  • hooks/useNotifications.tsx: Uint8Array を BufferSource に型キャスト
  • hooks/useNotifications.tsx: vibrate プロパティの型を unknown 経由でキャスト

5. web-push の型定義を追加

  • types/web-push.d.ts を新規作成
  • PushSubscription, WebPushError, 関数シグネチャを定義

6. lib/web-push.ts のエラー配列型を修正

  • subscription を non-null assertion で処理
  • error を unknown 型として扱う

テスト結果

✅ 型チェック: 成功
✅ ビルド: 成功
✅ ユニットテスト: 86/86 合格

すべてのCIチェックが成功する状態に修正しました。

## 修正内容

### 1. Supabase createClient関数のエクスポート
- lib/supabase.ts に createClient 関数を追加
- APIルートで使用できるように再エクスポート

### 2. 型安全性の改善
- app/api/push/send/route.ts: SubscriptionRow インターフェースを定義
- app/api/push/subscribe/route.ts: 型キャストを適切に実装
- app/gamemaster/page.tsx: filter/map関数の引数に明示的な型を追加
- app/chaser/page.tsx: map関数の引数に明示的な型を追加

### 3. テストの修正
- hooks/__tests__/useLocation.test.ts: resumeGame を削除(存在しない関数)
- hooks/__tests__/useLocationHistory.test.tsx: calculateDistance を lib/geometry から直接インポート

### 4. 通知APIの型問題を解決
- hooks/useNotifications.tsx: Uint8Array を BufferSource に型キャスト
- hooks/useNotifications.tsx: vibrate プロパティの型を unknown 経由でキャスト

### 5. web-push の型定義を追加
- types/web-push.d.ts を新規作成
- PushSubscription, WebPushError, 関数シグネチャを定義

### 6. lib/web-push.ts のエラー配列型を修正
- subscription を non-null assertion で処理
- error を unknown 型として扱う

## テスト結果
✅ 型チェック: 成功
✅ ビルド: 成功
✅ ユニットテスト: 86/86 合格

すべてのCIチェックが成功する状態に修正しました。
@at-ok at-ok merged commit ae02b50 into main Nov 20, 2025
4 checks passed
@at-ok at-ok deleted the claude/codebase-review-refactor-014RfXjs8uCCm3sMAL4vijND branch November 20, 2025 08:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants